home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 8714 < prev    next >
Encoding:
Text File  |  1996-08-05  |  1.6 KB  |  61 lines
  1. Path: news.halcyon.com!usenet
  2. From: george potts <solvris@halcyon.com>
  3. Newsgroups: comp.lang.c
  4. Subject: Re: revised code of calling a function twice from printf
  5. Date: Tue, 05 Mar 1996 16:46:28 -0800
  6. Organization: Solveris Inc
  7. Message-ID: <313CE064.6E6C@halcyon.com>
  8. References: <4hfs54$k4e@newsbf02.news.aol.com>
  9. NNTP-Posting-Host: blv-pm14-ip6.halcyon.com
  10. Mime-Version: 1.0
  11. Content-Type: text/plain; charset=us-ascii
  12. Content-Transfer-Encoding: 7bit
  13. X-Mailer: Mozilla 2.0 (WinNT; I)
  14.  
  15. Razine wrote:
  16. > 
  17. > Here is the actual code , modified with everyone's suggestions.  However
  18. > there is still a bug in here somewhere.  Can anyone please explain what is
  19. > wrong with this.
  20. > 
  21. > It returns
  22. > 
  23. > [1] NONE       [2] NE
  24. > 
  25. > I would like it to return
  26. > 
  27. > [1] NONE       [2] Asprin
  28. > 
  29. > Thank you very much for your help.  it is greatly appreciated.
  30. > 
  31. > #include <stdio.h>
  32. > #include <string.h>
  33. > 
  34. > char *display_drug_type(int drug_index);
  35. > 
  36. > int drug_inventory[5]={0,1,0,0,0};
  37. > 
  38. > int main() {
  39. > 
  40. >     printf("[1] %s             [2] %s
  41. > \n",display_drug_type(0),display_drug_type(1));
  42. > 
  43. > return 0;
  44. >         }
  45. > 
  46. > char *display_drug_type(int drug_index) {
  47. >    char drug_type[81]="\0";
  48. > 
  49. >    switch(drug_inventory[drug_index]) {
  50. >      case 0 : strcpy(drug_type,"NONE"); break;
  51. >      case 1 : strcpy(drug_type,"Asprin"); break;
  52. >      default : printf("Error in Display_drug_inventory");
  53. >                                                 }
  54. >    return drug_type;
  55. > }
  56.  
  57. you're returning a pointer to a character array that goes out of
  58. scope after the return. the memory could then be used for 
  59. something else before being displayed. move your drug_type 
  60. array.
  61.